Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a dialog for selecting/unselecting sections in reports #298

Closed
wants to merge 18 commits into from

Conversation

why-not-try-calmer
Copy link
Contributor

@why-not-try-calmer why-not-try-calmer commented Nov 20, 2023

This patch enables the user to select/unselect sections to include in reports. This should help decreasing the time to wait between each report.

image

Does it address #279, @spch-GL ?

@spch-GL
Copy link
Contributor

spch-GL commented Nov 20, 2023

Sur le principe quant à la sélection : oui
Mais dans le cas affiché, c'est difficile de voir si cela fonctionne

  • pour les cas spéciaux
  • pour les rapports hebdomadaires alors que le comptages sélectionné propose beaucoup de semaines (par exemple un comptage annuel)

@spch-GL
Copy link
Contributor

spch-GL commented Nov 20, 2023 via email

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Nov 20, 2023

Adrien, Dans le courriel reçu, il n’est pas fait mention que tu me poses une question. Est-ce normal ? ou un problème de paramétrage de github ? A+ Pierre Grellet Chef de projets en informatique technique (DDTE - SPCH - OSMO)

Pour que tu ne sois pas déranger par des notifications intempestives tout en recevant celles pertinentes je suggère d'utiliser ce réglage (Participating and mentions):

image

Il est probable que j'ai ajouté la question que tu n'as pas reçue en éditant le message en question. En allant directement dans le contexte tu dois normalement voir la discussion à jour. Je vais tâcher de ne pas éditer.

@why-not-try-calmer
Copy link
Contributor Author

Sur le principe quant à la sélection : oui Mais dans le cas affiché, c'est difficile de voir si cela fonctionne

  • pour les cas spéciaux
  • pour les rapports hebdomadaires alors que le comptages sélectionné propose beaucoup de semaines (par exemple un comptage annuel)

Okay, alors question: est-ce que section est un critère approprié à utiliser comme filtre dans tous ces cas? Il serait plus compliqué de changer de filtre en fonction du type de rapport, donc je cherche un filtre générique à tous ces cas.

@spch-GL
Copy link
Contributor

spch-GL commented Nov 20, 2023

Cela ne me dérange pas d'avoir plein d'alertes sur OpenComptage :-)
OK, alors je comprends.

@why-not-try-calmer why-not-try-calmer linked an issue Nov 20, 2023 that may be closed by this pull request
@spch-GL
Copy link
Contributor

spch-GL commented Nov 20, 2023

Dans un cas spécial, il y a plusieurs sections et potentiellement plusieurs rapports par section
Pour l'autre cas (cas standard), il n'y a que potentiellement plusieurs rapports (pour une seule et même section)

Donc il me semble qu'il manque la notion de rapports.
Qu'en penses-tu ?

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Nov 20, 2023

Dans un cas spécial, il y a plusieurs sections et potentiellement plusieurs rapports par section Pour l'autre cas (cas standard), il n'y a que potentiellement plusieurs rapports (pour une seule et même section)

Donc il me semble qu'il manque la notion de rapports. Qu'en penses-tu ?

Okay donc dans l'idéal tu voudrais:

cas standard:

  • rapport 1
  • rapport 2
  • ...

cas particulier:

  • section 1
    • rapport 1a
    • rapport 2b
  • section 2
    • rapport 2a
    • rapport 2b
    • ...
  • ...

@spch-GL
Copy link
Contributor

spch-GL commented Nov 20, 2023

oui
Et il me semble que la section peut rester aussi dans le cas standard car cela permet de tout sélectionner/désélectionner

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Nov 20, 2023

Encore une chose, est-il vrai que, pour les cas spéciaux, le nombre de rapports (1 par lundi) est le même pour toutes les sections, si bien que les sélecteurs subordonnés / imbriqués des rapports n'auraient pas besoin de varier d'une section à l'autre? Le code que j'ai en example suggère que c'est le cas, mais peut-être que j'ai omis quelque chose?

@spch-GL
Copy link
Contributor

spch-GL commented Nov 21, 2023

Oui, sauf en cas de problèmes de capteur/câblage.... ce qui ne devrait pas arriver souvent ! En fait, cela n'impacterait que les données de comptage, et au pire sur les sections impactées, les rapports seraient générés inutilement

Petite précision: Les rapports hebdomadaires concernent une semaine entière, et nos semaines commencent le lundi

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Nov 21, 2023

Ça semble fonctionner:

Capture.video.du.2023-11-21.14-08-53.webm

Mais je dois admettre que lorsque la liste est longue (plusieurs) sections ça peut devenir pénible de faire défiler. Un design alternatif serait:

sections:

  • section1
  • section2

jours:

  • jour1
  • jour2

en partant du principe que le résultat des sélections serait le produit cartésien sections x jours. Beaucoup plus facile à faire défiler mais on perd la finesse de grain (par jour / par section).

Une préférénce?

@spch-GL
Copy link
Contributor

spch-GL commented Nov 21, 2023

Je trouve que ca donne bien, merci

L'avantage avec la version de la vidéo, c'est que comme la section est parent des rapports (dont tu donnes le premier jour), il est facile de sélectionner tout ou rien puis d'aller améliorer la sélection. Du coup à mon avis les 2 boutons radio pourraient être supprimés.
Il y a au maximum 8 sections par cas spécial (avec notre matériel actuel...) mais le nombre de rapports possibles dépend des dates début et fin de traitement (comptage)

Comme améliorations, potentiellement dans un 2eme temps, je verrais :

  • présenter les sections dans des colonnes différentes
  • diminuer l'espace entre les lignes

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Nov 21, 2023

Je trouve que ca donne bien, merci

L'avantage avec la version de la vidéo, c'est que comme la section est parent des rapports (dont tu donnes le premier jour), il est facile de sélectionner tout ou rien puis d'aller améliorer la sélection. Du coup à mon avis les 2 boutons radio pourraient être supprimés. Il y a au maximum 8 sections par cas spécial (avec notre matériel actuel...) mais le nombre de rapports possibles dépend des dates début et fin de traitement (comptage)

Comme améliorations, potentiellement dans un 2eme temps, je verrais :

  • présenter les sections dans des colonnes différentes
  • diminuer l'espace entre les lignes

Merci de ton retour! Je pense qu'il faut quand même garder "select all/none" mais éventuellement ne pas les afficher lorsqu'il n'y a qu'une seule section. Car dans les cas où il y en a plus d'une, c'est assez pratique, non? (Après tout je fais une proposition qui doit marcher pour les rapports ordinaires et spéciaux.)

@spch-GL
Copy link
Contributor

spch-GL commented Nov 21, 2023

OK, et nous verrons à l'usage si nous préférons les garder ou pas

@why-not-try-calmer
Copy link
Contributor Author

OK, et nous verrons à l'usage si nous préférons les garder ou pas

Très, je mets de côté et je passe à la suite.

@why-not-try-calmer
Copy link
Contributor Author

Corrigé:

  • sous-checkboxes non mises à jours

@spch-GL
Copy link
Contributor

spch-GL commented Dec 4, 2023

Le nombre est bien mis à jour dans la boite de dialogue, mais aucun rapport n'est produit

@why-not-try-calmer why-not-try-calmer force-pushed the 279-select-reports branch 2 times, most recently from a58707e to 693f501 Compare December 5, 2023 09:30
@spch-GL
Copy link
Contributor

spch-GL commented Dec 5, 2023

Avec la version 2.1.3b, cela fonctionne et produit bien les rapports désirés
Reste l'ergonomie comme dit plus haut:

Comme améliorations, potentiellement dans un 2eme temps, je verrais :

présenter les sections dans des colonnes différentes
diminuer l'espace entre les lignes

@spch-GL
Copy link
Contributor

spch-GL commented Dec 5, 2023

Sur un cas particulier (55110005, 55120050, 64010085), en acceptant la proposition avec tous les rapports, il génère bien les 6 rapports attendu, mais il y a un petit bug dans les logs:
Voici le log Comptages qui est correct, avec 3 sections:

2023-12-05T12:00:42     INFO    2023-12-05 12:00:42.578387 - Generate report action started
2023-12-05T12:00:54     INFO    
              2023-12-05 12:00:54.702303 - Generate report action can really begin now for count 25 with file_path: L:/Devel/Inf_Technique/Comptage/OpenComptage/_tst_V211/RPT_heb.
              Selected sections and dates: {'55110005': [datetime.date(2022, 3, 14), datetime.date(2022, 3, 21)], '55120050': [datetime.date(2022, 3, 14), datetime.date(2022, 3, 21)], '64010085': [datetime.date(2022, 3, 14), datetime.date(2022, 3, 21)]}
              
2023-12-05T12:01:12     INFO    2023-12-05 12:01:12.905338 - Report generation RPT_heb ended
2023-12-05T12:01:12     INFO    2023-12-05 12:01:12.913337 - All tasks ended

Voici le log Report (nouveau), et il effectue le travail pour 4 sections...:

2023-12-05T12:00:54     INFO    Reporting on 4 sections
2023-12-05T12:00:54     INFO    Adding to workbook
2023-12-05T12:00:56     INFO    Adding to workbook
2023-12-05T12:00:58     INFO    Adding to workbook
2023-12-05T12:01:00     INFO    Adding to workbook
2023-12-05T12:01:04     INFO    Adding to workbook
2023-12-05T12:01:06     INFO    Adding to workbook
2023-12-05T12:01:08     INFO    Adding to workbook
2023-12-05T12:01:10     INFO    Adding to workbook

@why-not-try-calmer
Copy link
Contributor Author

Bien vu!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants